From: Sebastian Keller <skeller@gnome.org>
Date: Sun, 22 Oct 2023 23:06:59 +0000 (+0200)
Subject: calendar: Emit selected-date-changed before rebuilding the calendar
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~281^2~2^2~338
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=23bcff3348f63c5c10c8d3034665a55b747337c2;p=gnome-shell.git

calendar: Emit selected-date-changed before rebuilding the calendar

If the cursor was placed on one of the widgets that get removed when
rebuilding the calendar due to a month change, destroying the hovered
widget will trigger a repick. This repick can then trigger an allocation
while not all buttons of the calendar are present.

If the last allocation before selected-date-changed is emitted was from
such an incomplete state, DateMenuButton will still freeze the layout in
this state in its signal handler.

What freezing the layout in DateMenuButton is supposed to do is to
prevent size changes of the menu when changing days, but for this the
layout needs to be frozen before potentially rebuilding calendar. This
change ensures that by emitting the signal earlier.

Closes: https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/5411
Closes: https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/5469
Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2989>
---

diff --git a/js/ui/calendar.js b/js/ui/calendar.js
index 1371abf33..6de034c2c 100644
--- a/js/ui/calendar.js
+++ b/js/ui/calendar.js
@@ -471,11 +471,12 @@ export const Calendar = GObject.registerClass({
             return;
 
         this._selectedDate = date;
-        this._update();
 
         let datetime = GLib.DateTime.new_from_unix_local(
             this._selectedDate.getTime() / 1000);
         this.emit('selected-date-changed', datetime);
+
+        this._update();
     }
 
     updateTimeZone() {